-
Notifications
You must be signed in to change notification settings - Fork 304
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don't build unnecessary jars. #1786
Conversation
@AlexeyGy A long time has passed since the PR was put up, so maybe it's not even the same people working on the project anymore, is there a way to ping current reviewers? |
I think this might still be useful should we reopen it? |
Feel free to open a new PR with the code, I signed the CLA before, but we're not using this patch anymore. |
Happy to. Curious, why aren't you using the patch though? did you find it not helpful anymore? |
We've changed our workflows so the plugin is less integrated in them, that way issues with the plugin are less impactful, and we don't need to worry or work around them as much anymore. |
Checklist
Please note that the maintainers will not be reviewing this change until all checkboxes are ticked. See
the Contributions section in the README for more
details.
Discussion thread for this change
Issue number: #1785
Description of this change
Don't build the class jar if source jars are available, as we can use that in conjunction with the interface jar to index the code instead.